lib Fix Part 5/6 – Function.{apply, bind} #50453
Conversation
|
@typescript-bot run dt |
|
Heya @sandersn, I've started to run the diff-based user code test suite on this PR at 793adeb. You can monitor the build here. Update: The results are in! |
|
Heya @sandersn, I've started to run the diff-based top-repos suite on this PR at 793adeb. You can monitor the build here. Update: The results are in! |
|
@sandersn Here are the results of running the user test suite comparing Something interesting changed - please have a look. Details
|
|
@sandersn Here are the results of running the top-repos suite comparing Everything looks good! |
graphemecluster
changed the title
|
@typescript-bot run dt |
|
Heya @sandersn, I've started to run the diff-based user code test suite on this PR at f435fad. You can monitor the build here. Update: The results are in! |
|
Heya @sandersn, I've started to run the parallelized Definitely Typed test suite on this PR at f435fad. You can monitor the build here. Update: The results are in! |
|
Heya @sandersn, I've started to run the diff-based top-repos suite on this PR at f435fad. You can monitor the build here. Update: The results are in! |
|
@sandersn Here are the results of running the user test suite comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Something interesting changed - please have a look. Details
|
|
@sandersn Here are the results of running the top-repos suite comparing Everything looks good! |
|
Hey @sandersn, the results of running the DT tests are ready. |
|
The webpack failures are just a diff in the error message; the new message is better because there are fewer overloads. |
|
Addresses #52956 I think this looks good enough to ship without a design meeting discussion. One more test though: @typescript-bot run dt |
|
Heya @sandersn, I've started to run the parallelized Definitely Typed test suite on this PR at f435fad. You can monitor the build here. Update: The results are in! |
|
Argh I meant @typescript-bot test this |
|
Hey @sandersn, the results of running the DT tests are ready. |
|
Hm, I guess RWC didn't work. I think I'll merge this anyway. |
General Information
PR separated out from #49855, because there might be some members expecting smaller PRs. As mentioned in the comments from the big PR, whether to review a single, large PR or 6 smaller PRs is up to the TypeScript Team to decide. I couldn't have found a better way for this; hopefully this will not bring any trouble to the Team.
This PR partially addresses #49773.
For details and the track list about the changes, please see #49773.
Part 5/6, Function.{apply, bind}
The checker is now able to handle tuple with multiple variadics, thus this change.